Skip to content

Refactor A2A performance driver structure#139

Merged
MisterVVP merged 2 commits into
dev-add-performance-test-kit-for-sdkfrom
dev-refactor-a2a_performance_driver-for-maintainability
Jul 9, 2026
Merged

Refactor A2A performance driver structure#139
MisterVVP merged 2 commits into
dev-add-performance-test-kit-for-sdkfrom
dev-refactor-a2a_performance_driver-for-maintainability

Conversation

@MisterVVP

@MisterVVP MisterVVP commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Motivation

  • Make the in-process C++ performance driver easier to maintain by centralizing constants, scenario names, and common types.
  • Ensure measured latency does not include driver bookkeeping or contention from result collection.
  • Improve concurrency primitives and reuse SDK helpers instead of duplicating logic.

Description

  • Added tests/performance/a2a_performance_driver.h to centralize constants, scenario names/order, Options/ScenarioResult types, and function declarations.
  • Refactored tests/performance/a2a_performance_driver.cpp to use the new header, replaced string literals with named constexpr values, and moved shared helpers into the a2a::tests::performance namespace.
  • Replaced the push-delivery mutex and integer with a relaxed std::atomic<int> counter to avoid unnecessary locking on hot paths.
  • Replaced std::async + shared-result mutex with a fixed worker pool that uses std::atomic<int> work distribution and per-thread result buffers so aggregation happens after measured operations finish, avoiding measurement bias from collection locks.
  • Added tests/performance/README.md documenting the driver, measured fields, store backends, and example invocations.

@MisterVVP MisterVVP added the codex Open AI agent contribution label Jul 9, 2026 — with ChatGPT Codex Connector
@MisterVVP MisterVVP self-assigned this Jul 9, 2026
@MisterVVP MisterVVP merged commit a085868 into dev-add-performance-test-kit-for-sdk Jul 9, 2026
15 checks passed
@MisterVVP MisterVVP deleted the dev-refactor-a2a_performance_driver-for-maintainability branch July 9, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex Open AI agent contribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant